-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add GetDefaultTarget and SetDefaultTarget APIs #951
Add GetDefaultTarget and SetDefaultTarget APIs #951
Conversation
1327559
to
d1c0dab
Compare
d7b39b3
to
ef50d6d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally it looks great, some small nits posted
tests/tests/tier0/bluechi-get-default-target/test_bluechi_agent_get_default_target.py
Outdated
Show resolved
Hide resolved
Also please don't forget to update bluechictl man page: https://github.com/eclipse-bluechi/bluechi/blob/main/doc/man/bluechictl.1.md |
2e008ad
to
4b970c2
Compare
tests/tests/tier0/bluechictl-set-get-default-target/test_bluechictl_set_get_deault_target.py
Outdated
Show resolved
Hide resolved
tests/tests/tier0/bluechictl-set-get-default-target/test_bluechictl_set_get_deault_target.py
Outdated
Show resolved
Hide resolved
tests/tests/tier0/bluechictl-set-get-default-target/test_bluechictl_set_get_deault_target.py
Outdated
Show resolved
Hide resolved
4b970c2
to
7e28ac4
Compare
68a974d
to
b7f51c8
Compare
tests/tests/tier0/bluechictl-set-get-default-target/test_bluechictl_set_get_default_target.py
Outdated
Show resolved
Hide resolved
863766d
to
607f4a5
Compare
be1c18a
to
a39d66e
Compare
tests/tests/tier0/bluechictl-set-get-default-target/test_bluechictl_set_get_default_target.py
Outdated
Show resolved
Hide resolved
d4967b4
to
f6bbdb6
Compare
This feature will get and set the default target for a specific node. Solves eclipse-bluechi#944 Signed-off-by: Artiom Divak <[email protected]>
f6bbdb6
to
d974d1d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Fixes: #944
Signed-off-by: Artiom Divak [email protected]